From: Keir Fraser Date: Wed, 24 Jun 2009 10:05:22 +0000 (+0100) Subject: Scaling guest's TSC when the target machine's frequency is different X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~13701 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/success/%22http:/www.example.com/cgi/success?a=commitdiff_plain;h=6e10abac004556688ac79bf27a501401224b262d;p=xen.git Scaling guest's TSC when the target machine's frequency is different with its requirement. Using trap&emulate for guest's each rdtsc instruction first, maybe it can be optimized later. Signed-off-by: Xiantao Zhang --- diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c index e584e0c00b..a7f8cc1c54 100644 --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -144,26 +144,67 @@ uint8_t hvm_combine_hw_exceptions(uint8_t vec1, uint8_t vec2) return TRAP_double_fault; } +void hvm_enable_rdtsc_exiting(struct domain *d) +{ + struct vcpu *v; + + if ( opt_softtsc || !hvm_funcs.enable_rdtsc_exiting ) + return; + + for_each_vcpu ( d, v ) + hvm_funcs.enable_rdtsc_exiting(v); +} + +int hvm_gtsc_need_scale(struct domain *d) +{ + uint32_t gtsc_mhz, htsc_mhz; + + gtsc_mhz = d->arch.hvm_domain.gtsc_khz / 1000; + htsc_mhz = opt_softtsc ? 1000 : ((uint32_t)cpu_khz / 1000); + + d->arch.hvm_domain.tsc_scaled = (gtsc_mhz && (gtsc_mhz != htsc_mhz)); + return d->arch.hvm_domain.tsc_scaled; +} + +static u64 hvm_h2g_scale_tsc(struct vcpu *v, u64 host_tsc) +{ + uint32_t gtsc_khz, htsc_khz; + + if ( !v->domain->arch.hvm_domain.tsc_scaled ) + return host_tsc; + + htsc_khz = opt_softtsc ? 1000000 : cpu_khz; + gtsc_khz = v->domain->arch.hvm_domain.gtsc_khz; + return muldiv64(host_tsc, gtsc_khz, htsc_khz); +} + void hvm_set_guest_tsc(struct vcpu *v, u64 guest_tsc) { - u64 host_tsc; + uint64_t host_tsc, scaled_htsc; - rdtscll(host_tsc); + if ( opt_softtsc ) + host_tsc = hvm_get_guest_time(v); + else + rdtscll(host_tsc); + + scaled_htsc = hvm_h2g_scale_tsc(v, host_tsc); - v->arch.hvm_vcpu.cache_tsc_offset = guest_tsc - host_tsc; + v->arch.hvm_vcpu.cache_tsc_offset = guest_tsc - scaled_htsc; hvm_funcs.set_tsc_offset(v, v->arch.hvm_vcpu.cache_tsc_offset); } u64 hvm_get_guest_tsc(struct vcpu *v) { - u64 host_tsc; + uint64_t host_tsc, scaled_htsc; if ( opt_softtsc ) host_tsc = hvm_get_guest_time(v); else rdtscll(host_tsc); - return host_tsc + v->arch.hvm_vcpu.cache_tsc_offset; + scaled_htsc = hvm_h2g_scale_tsc(v, host_tsc); + + return scaled_htsc + v->arch.hvm_vcpu.cache_tsc_offset; } void hvm_migrate_timers(struct vcpu *v) diff --git a/xen/arch/x86/hvm/save.c b/xen/arch/x86/hvm/save.c index 235be64a96..a2c4a6ccc5 100644 --- a/xen/arch/x86/hvm/save.c +++ b/xen/arch/x86/hvm/save.c @@ -63,6 +63,15 @@ int arch_hvm_load(struct domain *d, struct hvm_save_header *hdr) /* Restore guest's preferred TSC frequency. */ d->arch.hvm_domain.gtsc_khz = hdr->gtsc_khz; + if ( hdr->gtsc_khz && hvm_gtsc_need_scale(d) ) + { + hvm_enable_rdtsc_exiting(d); + gdprintk(XENLOG_WARNING, "Loading VM(id:%d) expects freq: %dmHz, " + "but host's freq :%"PRIu64"mHz, trap and emulate rdtsc!!!\n", + d->domain_id, hdr->gtsc_khz / 1000, opt_softtsc ? 1000 : + cpu_khz / 1000); + } + /* VGA state is not saved/restored, so we nobble the cache. */ d->arch.hvm_domain.stdvga.cache = 0; diff --git a/xen/arch/x86/hvm/vmx/vmx.c b/xen/arch/x86/hvm/vmx/vmx.c index 72a324c7c3..4acbea3e0c 100644 --- a/xen/arch/x86/hvm/vmx/vmx.c +++ b/xen/arch/x86/hvm/vmx/vmx.c @@ -947,6 +947,14 @@ static void vmx_set_tsc_offset(struct vcpu *v, u64 offset) vmx_vmcs_exit(v); } +static void vmx_enable_rdtsc_exiting(struct vcpu *v) +{ + vmx_vmcs_enter(v); + v->arch.hvm_vmx.exec_control |= CPU_BASED_RDTSC_EXITING; + __vmwrite(CPU_BASED_VM_EXEC_CONTROL, v->arch.hvm_vmx.exec_control); + vmx_vmcs_exit(v); + } + void do_nmi(struct cpu_user_regs *); static void vmx_init_hypercall_page(struct domain *d, void *hypercall_page) @@ -1395,7 +1403,8 @@ static struct hvm_function_table vmx_function_table = { .msr_write_intercept = vmx_msr_write_intercept, .invlpg_intercept = vmx_invlpg_intercept, .set_uc_mode = vmx_set_uc_mode, - .set_info_guest = vmx_set_info_guest + .set_info_guest = vmx_set_info_guest, + .enable_rdtsc_exiting = vmx_enable_rdtsc_exiting }; static unsigned long *vpid_bitmap; diff --git a/xen/arch/x86/hvm/vpt.c b/xen/arch/x86/hvm/vpt.c index 071ef14517..6a12d37a21 100644 --- a/xen/arch/x86/hvm/vpt.c +++ b/xen/arch/x86/hvm/vpt.c @@ -33,7 +33,8 @@ void hvm_init_guest_time(struct domain *d) pl->stime_offset = -(u64)get_s_time(); pl->last_guest_time = 0; - d->arch.hvm_domain.gtsc_khz = cpu_khz; + d->arch.hvm_domain.gtsc_khz = opt_softtsc ? 1000000 : cpu_khz; + d->arch.hvm_domain.tsc_scaled = 0; } u64 hvm_get_guest_time(struct vcpu *v) diff --git a/xen/include/asm-x86/hvm/domain.h b/xen/include/asm-x86/hvm/domain.h index dc05b5aafe..5c7230487c 100644 --- a/xen/include/asm-x86/hvm/domain.h +++ b/xen/include/asm-x86/hvm/domain.h @@ -45,7 +45,7 @@ struct hvm_domain { struct hvm_ioreq_page buf_ioreq; uint32_t gtsc_khz; /* kHz */ - uint32_t pad0; + bool_t tsc_scaled; struct pl_time pl_time; struct hvm_io_handler io_handler; diff --git a/xen/include/asm-x86/hvm/hvm.h b/xen/include/asm-x86/hvm/hvm.h index b61c566094..d85a0f61c5 100644 --- a/xen/include/asm-x86/hvm/hvm.h +++ b/xen/include/asm-x86/hvm/hvm.h @@ -129,6 +129,7 @@ struct hvm_function_table { void (*invlpg_intercept)(unsigned long vaddr); void (*set_uc_mode)(struct vcpu *v); void (*set_info_guest)(struct vcpu *v); + void (*enable_rdtsc_exiting)(struct vcpu *v); }; extern struct hvm_function_table hvm_funcs; @@ -282,6 +283,9 @@ int hvm_event_needs_reinjection(uint8_t type, uint8_t vector); uint8_t hvm_combine_hw_exceptions(uint8_t vec1, uint8_t vec2); +void hvm_enable_rdtsc_exiting(struct domain *d); +int hvm_gtsc_need_scale(struct domain *d); + static inline int hvm_cpu_up(void) { if ( hvm_funcs.cpu_up )